Skip to content

Bump gqlparser v2.5.27 -> v2.5.28#7700

Merged
johanfylling merged 4 commits intoopen-policy-agent:mainfrom
robmyersrobmyers:7699_graphql_errorposf
Jun 18, 2025
Merged

Bump gqlparser v2.5.27 -> v2.5.28#7700
johanfylling merged 4 commits intoopen-policy-agent:mainfrom
robmyersrobmyers:7699_graphql_errorposf

Conversation

@robmyersrobmyers
Copy link
Copy Markdown
Contributor

Why the changes in this PR are needed?

The test case added here will verify that a crashing bug is fixed.

What are the changes in this PR?

Adds a test case to ensure that #7699 is fixed.

Notes to assist PR review:

Upstream issue: vektah/gqlparser#375
Upstream PR: vektah/gqlparser#376

Additional test case for invalid graphql schemas. open-policy-agent#7699

Signed-off-by: Rob Myers <1243316+robmyersrobmyers@users.noreply.github.com>
@robmyersrobmyers robmyersrobmyers force-pushed the 7699_graphql_errorposf branch from c8043a0 to f17d1f9 Compare June 16, 2025 09:21
@robmyersrobmyers robmyersrobmyers changed the title Additional test case for invalid graphql schemas. #7699 Bump gqlparser v2.5.27 -> v2.5.28 #7699 Jun 16, 2025
@robmyersrobmyers robmyersrobmyers changed the title Bump gqlparser v2.5.27 -> v2.5.28 #7699 Bump gqlparser v2.5.27 -> v2.5.28 Jun 16, 2025
@robmyersrobmyers
Copy link
Copy Markdown
Contributor Author

I think this is ready to merge. Even though it fixes a potential crashing bug, I don't think a patch release is needed. In my opinion, it can wait until 1.6.

Copy link
Copy Markdown
Contributor

@johanfylling johanfylling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

LGTM 👍

v1.6.0 will be released next week.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 16, 2025

Deploy Preview for openpolicyagent ready!

Name Link
🔨 Latest commit 026f99c
🔍 Latest deploy log https://app.netlify.com/projects/openpolicyagent/deploys/68519d0b8b96fa0008652fd5
😎 Deploy Preview https://deploy-preview-7700--openpolicyagent.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@johanfylling johanfylling merged commit 70b98df into open-policy-agent:main Jun 18, 2025
31 checks passed
@johanfylling johanfylling linked an issue Jun 23, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bump gqlparser dependency for ErrorPosf() fix

2 participants